Skip to content

Use a free port in u2m authentication flows rather than 8020. #1239

Merged
renaudhartert-db merged 5 commits into
mainfrom
renaud-hartert_data/randomport
Jul 20, 2025
Merged

Use a free port in u2m authentication flows rather than 8020. #1239
renaudhartert-db merged 5 commits into
mainfrom
renaud-hartert_data/randomport

Conversation

@renaudhartert-db

Copy link
Copy Markdown
Contributor

What changes are proposed in this pull request?

This PR modifies the U2M flow so that the listener server uses a port that is free rather than the hardcoded 8020.

How is this tested?

Added a unit tests to verify that different ports are assigned to different instances of PersistentAuth.

@renaudhartert-db renaudhartert-db changed the title Renaud hartert data/randomport Use free ports in u2m authentication flows rather than 8020. Jun 26, 2025
@renaudhartert-db renaudhartert-db changed the title Use free ports in u2m authentication flows rather than 8020. Use a free port in u2m authentication flows rather than 8020. Jun 26, 2025
@github-actions

Copy link
Copy Markdown

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-go

Inputs:

  • PR number: 1239
  • Commit SHA: 28527e1cca8f2ce183e21548f8051b86e0377598

Checks will be approved automatically on success.

@renaudhartert-db renaudhartert-db added this pull request to the merge queue Jul 20, 2025
Merged via the queue into main with commit 667c53b Jul 20, 2025
15 checks passed
@renaudhartert-db renaudhartert-db deleted the renaud-hartert_data/randomport branch July 20, 2025 14:41
shreyas-goenka added a commit that referenced this pull request Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants